Skip to content

fix: пояснить черновики в распределении проектов экспертам - #447

Merged
TThanos3000 merged 1 commit into
devfrom
fix/dev-expert-draft-explanation-20261018
Oct 18, 2026
Merged

TThanos3000 merged 1 commit into
devfrom
fix/dev-expert-draft-explanation-20261018

Conversation

@TThanos3000

@TThanos3000 TThanos3000 commented Oct 18, 2026 •

Copy link
Copy Markdown
Contributor

Что изменено

В списке экспертов и предпросмотре распределения видны сданные проекты, которые всё ещё являются черновиками и недоступны к оценке. Пустой план теперь относится к доступным проектам, поэтому не противоречит счётчику нехватки.

Зачем

На dev проект №232 имеет submitted=True и draft=True. Счётчик показывал нехватку эксперта, а распределение пропускало черновик и ошибочно сообщало, что эксперты уже выбраны для всех проектов.

Реализация

Optional unavailableProjects model, пояснение с именами проектов в summary и auto/preview; сохранены текущие формы, действия и layout.

API

Совместимое optional поле unavailable_projects из API PR PROCOLLAB-github/api#790. Payload и eligibility не менялись. Без поля остаётся прежний интерфейс, после появления API показывает пояснение.

UI

Существующий DS warning, Mont и модалка по центру. Черновик не публикуется и не назначается автоматически.

Permissions

Не менялись: имена доступны только менеджеру своей программы; решение о публикации остаётся у автора, backend ограничения сохранены.

Проверки

  • 13 tests компонента, включая исключённый черновик в summary и пустом плане, disabled apply.
  • Angular development build, Prettier/ESLint затронутых файлов и diff --check.
  • 9 browser checks на реальном локальном Django/PostgreSQL, включая active/inactive дубль и submitted draft, 1440/390/360 px; без JS errors.
  • Checked by source: review contract fallback, отсутствие изменения permissions, payload, публикации и назначения.

Acceptance criteria

  • Недоступный проект назван и объяснён.
  • Пустой план не заявляет, что состав экспертов выбран для недоступных проектов.
  • Опубликованные проекты распределяются по прежним правилам; черновики не публикуются автоматически.

Не проверено

Live пояснение для №232 — после backend-first rollout. Внешние письма не отправлялись агентом.

Риски / migration

Миграций нет. Optional поле позволяет независимые dev-выкладки: старый API не ломает новый UI, старый UI игнорирует дополнительное поле. Итоговый smoke требуется после обеих выкладок.

@TThanos3000
TThanos3000 merged commit e1e21b3 into dev Oct 18, 2026
@TThanos3000
TThanos3000 deleted the fix/dev-expert-draft-explanation-20261018 branch October 18, 2026 11:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants